Skip to content

[#530] Fix OG image — match home page moleskine layout#533

Merged
realproject7 merged 2 commits intomainfrom
task/530-og-image-match-home
Mar 25, 2026
Merged

[#530] Fix OG image — match home page moleskine layout#533
realproject7 merged 2 commits intomainfrom
task/530-og-image-match-home

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

  • Redesigned OG image from notebook-left + metadata-right to single centered moleskine card
  • Layout matches home page StoryCard:
    • Top-left inside: genre tag
    • Center inside: story title
    • Bottom inside: plot count + TVL with USD value
    • Below moleskine: author name + plotlink.xyz branding
  • Added TVL with USD display (via getPlotUsdPrice + formatUsdValue)
  • Bumped version to 0.1.8

Test plan

  • Visit /story/[id]/og — image shows centered moleskine card
  • Genre tag at top-left of card
  • Title centered in card
  • Plot count + TVL at bottom of card
  • Author name + plotlink.xyz below card
  • Image renders at 1200x630 (OG standard)

Fixes #530

🤖 Generated with Claude Code

- Redesign to single centered moleskine card (was notebook left + metadata right)
- Top-left inside: genre tag
- Center inside: story title
- Bottom inside: plot count + TVL with USD value
- Below moleskine: author name + plotlink.xyz branding
- Bump version to 0.1.8

Fixes #530

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
plotlink Ignored Ignored Mar 25, 2026 10:07am

Request Review

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

APPROVE — major simplification of the OG image layout. Centered moleskine card matches the home page StoryCard layout. TVL with USD display added cleanly. Net -76 lines. No issues.

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: REQUEST CHANGES

Summary

The OG layout is much closer, but it still does not match the home page moleskine card layout exactly because the notebook cover was changed into a square tile.

Findings

  • [medium] The cover no longer uses the StoryCard's portrait notebook proportions.
    • File: src/app/story/[storylineId]/og/route.tsx:106
    • Suggestion: Keep the cover in the same portrait 2:3 shape as the home page StoryCard and scale that notebook within the 1200x630 canvas instead of using a 440x440 square.

Decision

Requesting changes because issue #530 explicitly says to match the home page moleskine layout exactly, and the current 440x440 cover still diverges from that reference.

Changed OG moleskine from 440x440 square to 360x540 (2:3 aspect-ratio).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Re-reviewed — moleskine now 360×540 (2:3 portrait ratio), matching home page aspect-[2/3]. APPROVE.

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: APPROVE

Summary

The updated head fixes the remaining layout mismatch by restoring the notebook to a portrait 2:3 aspect while keeping the centered StoryCard-style composition.

Findings

  • No blocking findings.

Decision

Approve. The current OG image now matches issue #530's requested home page moleskine layout closely enough to satisfy the acceptance criteria.

@realproject7 realproject7 merged commit 62f4a6b into main Mar 25, 2026
5 checks passed
@realproject7 realproject7 deleted the task/530-og-image-match-home branch March 25, 2026 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix OG image — match home page moleskine layout exactly

2 participants